Skip to content

Update Persian translations (fa, fa_IR) for Django application#897

Merged
Andrew-Chen-Wang merged 2 commits intojazzband:masterfrom
mahdirahimi1999:feature/fa-translations
Mar 28, 2025
Merged

Update Persian translations (fa, fa_IR) for Django application#897
Andrew-Chen-Wang merged 2 commits intojazzband:masterfrom
mahdirahimi1999:feature/fa-translations

Conversation

@mahdirahimi1999
Copy link
Copy Markdown
Member

This pull request updates the Persian translations for the djangorestframework-simplejwt project, adding support for both the "fa" and "fa_IR" languages. The translations include various messages related to token handling, authentication, and errors. This update aims to improve the localization for Persian-speaking users using the package.

Changes:

  • Added translations for "fa" and "fa_IR" locales
  • Included error messages and authentication-related strings
  • Updated .po files to reflect the latest translations

This update ensures better localization support and enhances the user experience for Persian-speaking users.

Please review and merge the changes to improve localization for the project.

@mahdirahimi1999
Copy link
Copy Markdown
Member Author

Please check this PR. @Andrew-Chen-Wang

#, python-format
msgid "Blacklisted token for %(user)s"
msgstr ""
msgstr "توکن لیست سیاه برای %(user)s"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi is this using Google Translate? Persian should be right to left

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, no, it's a manual translation. Persian is RTL, but PO files handle it correctly.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you have a link or proof of this in a demo app? send a picture

Comment thread rest_framework_simplejwt/locale/fa_IR/LC_MESSAGES/django.po Outdated
Comment thread rest_framework_simplejwt/locale/fa_IR/LC_MESSAGES/django.po Outdated
Comment thread rest_framework_simplejwt/locale/fa_IR/LC_MESSAGES/django.po Outdated
#| msgid "No active account found with the given credentials"
msgid "No active account found for the given token."
msgstr "هیچ اکانت فعالی برای اطلاعات داده شده یافت نشد"
msgstr "هیچ اکانت فعالی برای توکن داده شده یافت نشد."
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is there a period at the end?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why have you marked this as resolved? Periods go at the left, not right, correct?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, I adjusted the translation for accuracy. In Persian, punctuation follows the text, so the period is correct.

Copy link
Copy Markdown
Member

@Andrew-Chen-Wang Andrew-Chen-Wang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx for the PR and thx for clarifying concerns in the comments!

@mahdirahimi1999
Copy link
Copy Markdown
Member Author

You're welcome! Glad I could help, and thanks for reviewing!

@mahdirahimi1999
Copy link
Copy Markdown
Member Author

Will this PR be merged? If any changes are needed, please let me know so I can make them. @Andrew-Chen-Wang

@Andrew-Chen-Wang Andrew-Chen-Wang merged commit acb1483 into jazzband:master Mar 28, 2025
23 checks passed
@mahdirahimi1999 mahdirahimi1999 deleted the feature/fa-translations branch March 28, 2025 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants